Skip to content

feat(plan-review): add plan completeness review plugin#64

Merged
cblecker merged 1 commit into
mainfrom
feat/plan-review-plugin
Jul 3, 2026
Merged

feat(plan-review): add plan completeness review plugin#64
cblecker merged 1 commit into
mainfrom
feat/plan-review-plugin

Conversation

@cblecker

@cblecker cblecker commented Jul 3, 2026 •
edited by coderabbitai Bot
Loading

Copy link
Copy Markdown
Owner

Summary

  • Adds a new plan-review plugin with a finalize-plan skill (/plan-review:finalize-plan)
  • The skill reviews plan files before accepting them with showClearContextOnPlanAccept, ensuring key decisions, file names, commands, and constraints from the conversation are captured so a fresh context can execute faithfully
  • Lightweight by design — a focused prompt, not a procedure manual

Test plan

  • claude plugin validate ./plan-review passes
  • claude plugin validate . passes
  • Markdown linting passes
  • Enter plan mode, invoke /plan-review:finalize-plan, confirm it reviews the plan and reports gaps
  • Invoke outside plan mode, confirm it exits early with a message

Summary by CodeRabbit

  • New Features

    • Added a new plan-review plugin to the available plugins list and plugin metadata.
    • Introduced a new planning workflow step to help review and finalize plan files before context is cleared.

  • Documentation

    • Updated the plugin catalog in the README to describe the new plan-review option and what it does.

Copilot AI review requested due to automatic review settings July 3, 2026 17:46
@coderabbitai

coderabbitai Bot commented Jul 3, 2026 •
edited
Loading

Copy link
Copy Markdown

Review Change Stack

Warning

Review limit reached

@cblecker, you've reached your PR review limit, so we couldn't start this review.

Next review available in: 41 minutes

Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available.
You're only billed for reviews past your plan's rate limits ($0.25/file).

How can I continue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews.

How do review limits work?

CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability.

For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window.

Please refer docs for additional details.

Review details
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 35d3cbe1-574d-4d36-9b2b-f2ae4afe41d7

📥 Commits

Reviewing files that changed from the base of the PR and between ee3cd66 and 7d42401.

📒 Files selected for processing (5)
  • .claude-plugin/marketplace.json
  • README.md
  • plan-review/.claude-plugin/plugin.json
  • plan-review/README.md
  • plan-review/skills/finalize-plan/SKILL.md
📝 Walkthrough

Walkthrough

This PR adds a new plan-review plugin: registers it in marketplace.json and README.md, adds plugin.json metadata, and introduces a finalize-plan skill (SKILL.md) that verifies plan files are complete before context is cleared.

Changes

Plan-review plugin addition

Layer / File(s) Summary
Plugin metadata and skill definition
plan-review/.claude-plugin/plugin.json, plan-review/skills/finalize-plan/SKILL.md		 Adds plugin schema metadata (name, version, description, author) and a finalize-plan skill checklist for auditing plan files against conversation context.
Marketplace and README registration
.claude-plugin/marketplace.json, README.md		 Registers the plan-review plugin entry in the marketplace plugins array and adds a corresponding row to the README plugins table.

Estimated code review effort: 1 (Trivial) | ~3 minutes

Poem

A rabbit checks the plan once more,
before the context hits the floor,
new plugin joins the marketplace row,
finalize-plan makes sure we know,
each name, each fix, each command clear—
no missing piece to bring us fear. 🐇

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly matches the main change: adding a plan-review plugin for checking plan completeness.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/plan-review-plugin

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

Copilot AI reviewed Jul 3, 2026
View reviewed changes

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds a new lightweight plan-review plugin to the marketplace. Its single finalize-plan skill (/plan-review:finalize-plan) runs during plan mode to check that conversation-specific decisions, names, commands, and constraints are captured in the plan file so a fresh context can execute it faithfully. The change follows the documented "Adding a Plugin" flow (plugin manifest, marketplace entry, root README table row).

Changes:

  • New plan-review plugin manifest (plugin.json) starting at version 0.1.0.
  • New prompt-only finalize-plan skill that gates on plan mode, reviews the plan for completeness, and reports gaps.
  • Registration in .claude-plugin/marketplace.json and the root README.md Available Plugins table.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File Description
.claude-plugin/marketplace.json Adds the plan-review entry, consistent with existing entries and the documented format.
plan-review/.claude-plugin/plugin.json New plugin manifest with core fields; no plugin-level README accompanies it, unlike every other plugin.
plan-review/skills/finalize-plan/SKILL.md New prompt-only skill defining plan-mode gating, review criteria, and reporting format.
README.md Adds the plan-review row to the Available Plugins table.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread plan-review/.claude-plugin/plugin.json
Comment on lines +3 to +5
"name": "plan-review",
"version": "0.1.0",
"description": "Pre-flight review of plan files before context clear",
@cblecker
feat(plan-review): add plan completeness review plugin
7d42401 
New plugin with a finalize-plan skill that reviews plan files before
context clear to ensure key decisions, commands, and constraints from
the conversation are captured.

Assisted-by: Claude:claude-opus-4-6
@cblecker cblecker force-pushed the feat/plan-review-plugin branch from ee3cd66 to 7d42401 Compare July 3, 2026 18:05
@cblecker cblecker merged commit c4c2d91 into main Jul 3, 2026
13 checks passed
@cblecker cblecker deleted the feat/plan-review-plugin branch July 3, 2026 18:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@cblecker